Skip to content

Resolve snowflake deprecations in tests#40398

Merged
potiuk merged 3 commits into
apache:mainfrom
dirrao:snowflake_deprecations
Jun 24, 2024
Merged

Resolve snowflake deprecations in tests#40398
potiuk merged 3 commits into
apache:mainfrom
dirrao:snowflake_deprecations

Conversation

@dirrao

@dirrao dirrao commented Jun 24, 2024

Copy link
Copy Markdown
Contributor

Resolve snowflake deprecations in tests
related: #38642

@boring-cyborg boring-cyborg Bot added area:providers provider:snowflake Issues related to Snowflake provider labels Jun 24, 2024
@dirrao dirrao requested review from Taragolis and hussein-awala and removed request for Taragolis June 24, 2024 08:43
@dirrao

dirrao commented Jun 24, 2024

Copy link
Copy Markdown
Contributor Author

We are running the tests on _process_output in SnowflakeOperator. Do we need to retain this test case as we are deprecating the SnowflakeOperator? Or Adjust it as per the default _process_output from SQLExecuteQueryOperator?

@uranusjr

Copy link
Copy Markdown
Member

We probably should try to retain to test. Not quite sure.

@dirrao dirrao force-pushed the snowflake_deprecations branch from 34a95af to 7726780 Compare June 24, 2024 12:11
@dirrao dirrao requested a review from potiuk June 24, 2024 16:24
@potiuk potiuk merged commit 67e1ce6 into apache:main Jun 24, 2024
romsharon98 pushed a commit to romsharon98/airflow that referenced this pull request Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:snowflake Issues related to Snowflake provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants